Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reword F23 #1826

Merged
merged 7 commits into from
Jan 12, 2024
Merged

Reword F23 #1826

merged 7 commits into from
Jan 12, 2024

Conversation

patrickhlauke
Copy link
Member

@patrickhlauke patrickhlauke commented May 21, 2021

in line with the proposed clarification for 1.4.2 #1825 this clarifies that a mechanism needs to be available. the mechanism can actually be provided by the UA/system, not necessarily by the Web page. Key here is that it must be possible to turn off the sound independently from any other audio output.

Closes #1543

Only makes sense to merge if #1825 (or an equivalent/modified version of the PR) is also merged (Completed)

in line with the proposed clarification for 1.4.2 #1825 this clarifies that a *mechanism* needs to be available. the mechanism can actually be provided by the UA/system, not necessarily by the Web page. Key here is that it must be possible to turn off the sound *independently* from any other audio output.

Closes #1543

Only makes sense to merge if #1825 (or an equivalent/modified version of the PR) is also merged
@patrickhlauke
Copy link
Member Author

Any news on this @alastc ?

@patrickhlauke
Copy link
Member Author

bump

@mbgower mbgower self-assigned this Dec 4, 2023
@mbgower
Copy link
Contributor

mbgower commented Dec 5, 2023

@patrickhlauke you stated

Only makes sense to merge if #1825 (or an equivalent/modified version of the PR) is also merged

That was closed due to lack of support to change the normative text. However, there was a minor tweak to the Understanding document that was merged. Do you consider that sufficient to support the incorporation of this technique correction?

@bruce-usab
Copy link
Contributor

Discussed on call 12/8, minor edit to track closer to SC phrasing.

@alastc
Copy link
Contributor

alastc commented Dec 8, 2023

@mbgower you wanted to quickly scan this one before putting out to the group.

@mbgower mbgower removed the Editorial label Dec 8, 2023
<li> Check that there is a way in a Web page to turn off any sound that
plays automatically for more than three seconds.</li>
<li>Check that there is a mechanism to turn off any sound that
plays automatically for more than three seconds, independently from the overall system volume level.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So if the volume is the same as the system volume does the length of an auto-played audio file matter? I would hope that anything automatically played you could shut off after 3 minutes (regardless of the volume level).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think you're missing the point here - the "independently..." refers to the mechanism to turn off any sound. i.e the mechanism isn't "just turn off the audio of your entire system ... there ... now you satisfy the SC".

however, if this is confusing (that the "independently" refers to the mechanism), can be turned around to something like

Check that there is a mechanism, independent from the overall system volume level, to turn off any sound that plays automatically for more than three seconds

@mraccess77
Copy link

Does this now mean that if checking conformance with a browser like Chrome this SC would be always met?

<h2>Description</h2>
<p>This describes a failure condition for Success Criteria involving sound. If sound does not
turn off automatically within 3 seconds and there is no way to turn the
sound off, independently from the overall system volume level, then Success Criterion 1.4.2 would not be met.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
sound off, independently from the overall system volume level, then Success Criterion 1.4.2 would not be met.
sound off, or control the volume independently from the overall system volume level, then Success Criterion 1.4.2 would not be met.

<section class="procedure"><h3>Procedure</h3>
<ol>
<li> Check that there is a way in a Web page to turn off any sound that
plays automatically for more than three seconds.</li>
<li>Check that there is a mechanism, independent from the overall system volume control, to turn off any sound that
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<li>Check that there is a mechanism, independent from the overall system volume control, to turn off any sound that
<li>Check that there is a mechanism, independent from the overall system volume control, to turn off or turn down any sound that

@mbgower
Copy link
Contributor

mbgower commented Dec 13, 2023

Does this now mean that if checking conformance with a browser like Chrome this SC would be always met?

@mraccess77 Is it your sense that changing the failure technique has altered the SC requirement? Or is it that as a result of re-assessing 1.4.2 during this review, are you identifying a possible way to meet it? I'm just trying to decide if you are identifying a concern with this correction.

@patrickhlauke
Copy link
Member Author

patrickhlauke commented Dec 13, 2023

@mraccess77 i believe that was the long discussion that was had which then led to this PR (though i admit that because this has been in limbo so long, I might only be half remembering it).

I believe the discussion started with #1533 (though I also see #1543)

we then had various split off/parallel attempts to address this like #1825
there was discussions here https://www.w3.org/2022/08/16-ag-minutes.html#item10

I think this PR here was the final outcome after all of that, in my mind.

in direct answer "so does it mean muting tabs is now ok", i think back in those discussions we came to the conclusion that only if that's available on all (target) platforms / an accessibility-supported question in the end

@mraccess77
Copy link

I believe this does not change the SC but the practice of how/when people fail this issue. That is - it changes the interpretation that most people had. This is fine -but I want to make sure that the group is alignment that this interpretation is indeed correct and the result should be for many to no longer fail this SC most of the time when audio is playing automatically without other page controls. I believe this reading is in line with Trusted Tester.

</section><section id="examples"><h2>Examples</h2>
</section>
<section id="description">
<h2>Description</h2>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's important to mention that most browsers use heuristics to mute automatically playing audio / video now. That's probably not sufficient as a mechanism, at least that's what ACT concluded when we looked at it. Similarly, several browser support muting individual tabs, we didn't think that qualifies as a mechanism either. I think its important to make that clear.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I completely agree.
In addition, some platforms offer the possibility to mute specific browser windows. I would neither consider this a valid mechanism.
In the initial issue description, the phrase "turn off the sound independently from any other audio output." was used. I think this is much closer to what we want to achieve.

@mbgower
Copy link
Contributor

mbgower commented Jan 8, 2024

IMO, there are enough comments about what is considered an acceptable mechanism that that should be opened as a separate issue/discussion. I believe that discussion can take place independent of accepting this PR, and have opened #3623

@mbgower mbgower merged commit cdd703a into main Jan 12, 2024
1 check passed
@mbgower mbgower deleted the patrickhlauke-issue1543 branch January 15, 2024 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failure F23 is NOT a failure if you look at the Procedure
8 participants